Skip to content

fix(source-zendesk-sunshine): update CDK to prerelease version with OAuth race condition fix #72384

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
devin-ai-integration wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix(source-zendesk-sunshine): update CDK to prerelease version with OAuth race condition fix #72384

devin-ai-integration wants to merge 2 commits into from
+3 −2

Conversation

@devin-ai-integration
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Jan 27, 2026

What

Updates the source-zendesk-sunshine connector to use a prerelease CDK version that fixes an OAuth race condition where multiple streams starting concurrently with an expired token would all attempt to refresh simultaneously, causing failures with single-use refresh tokens.

Related CDK PR: airbytehq/airbyte-python-cdk#883

How

  1. Updates baseImage in metadata.yaml from stable 7.6.5 to prerelease 7.6.5.post3.dev21398747886 which includes thread-safe token refresh with double-checked locking
  2. Bumps dockerImageTag from 0.3.41 to 0.3.42
  3. Adds changelog entry documenting the fix

Review guide

  1. airbyte-integrations/connectors/source-zendesk-sunshine/metadata.yaml - CDK version update and version bump
  2. docs/integrations/sources/zendesk-sunshine.md - Changelog entry

User Impact

Users authenticating via OAuth should no longer experience authentication failures when multiple streams start concurrently with an expired token.

Can this PR be safely reverted and rolled back?

  • YES 💚

Requested by: gl_anatolii.yatsuk@airbyte.io

Link to Devin run: https://app.devin.ai/sessions/2f784e9440324787893047ac3a1c0fbc

@devin-ai-integration
fix(source-zendesk-sunshine): update CDK to prerelease version with O…
3e34670 
…Auth race condition fix

Co-Authored-By: gl_anatolii.yatsuk@airbyte.io <gl_anatolii.yatsuk@airbyte.io>
@devin-ai-integration
Copy link
Contributor Author

devin-ai-integration bot commented Jan 27, 2026

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions
Copy link
Contributor

github-actions bot commented Jan 27, 2026

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

💡 Show Tips and Tricks

PR Slash Commands

Airbyte Maintainers (that's you!) can execute the following slash commands on your PR:

  • 🛠️ Quick Fixes
    • /format-fix - Fixes most formatting issues.
    • /bump-version - Bumps connector versions, scraping changelog description from the PR title.
  • ❇️ AI Testing and Review:
    • /ai-docs-review - Provides AI-powered documentation recommendations for PRs with connector changes.
    • /ai-prove-fix - Runs prerelease readiness checks, including testing against customer connections.
    • /ai-canary-prerelease - Rolls out prerelease to 5-10 connections for canary testing.
    • /ai-review - AI-powered PR review for connector safety and quality gates.
  • 🚀 Connector Releases:
    • /publish-connectors-prerelease - Publishes pre-release connector builds (tagged as {version}-preview.{git-sha}) for all modified connectors in the PR.
    • /bump-progressive-rollout-version - Bumps connector version with an RC suffix (2.16.10-rc.1) for progressive rollouts (enableProgressiveRollout: true).
      • Example: /bump-progressive-rollout-version changelog="Add new feature for progressive rollout"
  • ☕️ JVM connectors:
    • /update-connector-cdk-version connector=<CONNECTOR_NAME> - Updates the specified connector to the latest CDK version.
      Example: /update-connector-cdk-version connector=destination-bigquery
    • /bump-bulk-cdk-version bump=patch changelog='foo' - Bump the Bulk CDK's version. bump can be major/minor/patch.
  • 🐍 Python connectors:
    • /poe connector source-example lock - Run the Poe lock task on the source-example connector, committing the results back to the branch.
    • /poe source example lock - Alias for /poe connector source-example lock.
    • /poe source example use-cdk-branch my/branch - Pin the source-example CDK reference to the branch name specified.
    • /poe source example use-cdk-latest - Update the source-example CDK dependency to the latest available version.
  • ⚙️ Admin commands:
    • /force-merge reason="<REASON>" - Force merges the PR using admin privileges, bypassing CI checks. Requires a reason.
      Example: /force-merge reason="CI is flaky, tests pass locally"
📚 Show Repo Guidance

Helpful Resources

📝 Edit this welcome message.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 27, 2026 •
edited
Loading

source-zendesk-sunshine Connector Test Results

9 tests   3 ✅  11s ⏱️
1 suites  6 💤
1 files    0 ❌

Results for commit 16ba2fa.

♻️ This comment has been updated with latest results.

@tolik0 tolik0 self-assigned this Jan 27, 2026
@tolik0 tolik0 marked this pull request as ready for review January 27, 2026 15:29
@tolik0 tolik0 requested a review from agarctfi January 27, 2026 15:30
@devin-ai-integration
chore: trigger docs rebuild
16ba2fa 
Co-Authored-By: gl_anatolii.yatsuk@airbyte.io <gl_anatolii.yatsuk@airbyte.io>
@github-actions
Copy link
Contributor

github-actions bot commented Jan 27, 2026

Deploy preview for airbyte-docs ready!

✅ Preview
https://airbyte-docs-bstjp16so-airbyte-growth.vercel.app

Built with commit 16ba2fa.
This pull request is being automatically deployed with vercel-action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@agarctfi agarctfi Awaiting requested review from agarctfi

At least 1 approving review is required to merge this pull request.

Assignees

@tolik0 tolik0

Labels

connectors/source/zendesk-sunshine

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tolik0 @octavia-squidington-iii